Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: use drag.* events to set drop target styles #91

Merged
merged 1 commit into from
Aug 9, 2023

Conversation

pinussilvestrus
Copy link
Contributor

@pinussilvestrus pinussilvestrus commented Jul 20, 2023

Closes #53

This is to make this behavior agnostic from the dragging library we use under the hood but use our own internal events. The styles of the border is unchanged.

Demo: https://53-replace-dragula-behavior--camunda-form-playground.netlify.app/

image

src/components/PlaygroundComponent.js Outdated Show resolved Hide resolved
src/components/PlaygroundComponent.js Outdated Show resolved Hide resolved
@pinussilvestrus
Copy link
Contributor Author

Ready for review again.

@pinussilvestrus
Copy link
Contributor Author

@slolatte @esraagamal6 this is ready for test now 👍 https://53-replace-dragula-behavior--camunda-form-playground.netlify.app/

  • The dashed border for the drop area should be visible on larger screens when the preview is closed
  • On smaller screens and when the preview is open, it should not be visible

@esraagamal6
Copy link

@pinussilvestrus Acceptance testing passed

@pinussilvestrus pinussilvestrus merged commit 8e783b9 into main Aug 9, 2023
8 checks passed
@pinussilvestrus pinussilvestrus deleted the 53-replace-dragula-behavior branch August 9, 2023 11:31
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update how the drop target is highlighted
3 participants